Skip to content

Wide-card layout for holding cards at low lot counts (#52)#54

Merged
heyitsStylez merged 1 commit intomainfrom
wide-holding-cards
May 7, 2026
Merged

Wide-card layout for holding cards at low lot counts (#52)#54
heyitsStylez merged 1 commit intomainfrom
wide-holding-cards

Conversation

@heyitsStylez
Copy link
Copy Markdown
Owner

Closes #52.

Summary

  • holdings-grid gains holdings-grid--wide class when ≤ 2 visible open lots (asset filter honored).
  • @media (min-width:720px) rule turns each .hcard into a 4-column horizontal row (Identity · Net Cost · Spot · Footer stats). Below 720px the existing vertical card grid is unchanged.
  • Spot column always renders; missing price → muted Spot — placeholder with spot unavailable sub-line so the layout doesn't reflow when prices arrive.

Test plan

  • python3 build.py --check passes
  • npm test passes (85 tests, +6 new in test/integration/holding-card-wide.test.js)
  • Manual: at ≥720px viewport, 1 lot → single full-width row; 2 lots → two stacked rows; 3 lots → existing tile grid
  • Manual: shrink viewport <720px → vertical grid regardless of lot count
  • Manual: asset filter (e.g. BTC with 1 BTC + N other lots) → wide layout
  • Manual: edit btn, lot badge, merge-lots button still functional in wide mode

🤖 Generated with Claude Code

Adds holdings-grid--wide class when <=2 visible open lots, activating a
4-column horizontal row layout at >=720px viewport width via CSS media
query. Asset filter narrows the visible-lot count for the threshold.
Spot column always renders; falls back to a stable "Spot —" placeholder
with "spot unavailable" sub-line when livePrices[asset] is missing, so
the layout doesn't reflow when prices arrive.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented May 7, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
hyperwheel Ready Ready Preview, Comment May 7, 2026 10:02am

@heyitsStylez heyitsStylez merged commit 08db61a into main May 7, 2026
4 checks passed
@heyitsStylez heyitsStylez deleted the wide-holding-cards branch May 7, 2026 10:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Wide-card layout for holding cards at low lot counts

1 participant